-
Notifications
You must be signed in to change notification settings - Fork 1
Rewrite the action as a javascript action #91
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
d1aaa83
to
184916f
Compare
184916f
to
8a16313
Compare
48c11be
to
a66856f
Compare
github-token: | ||
description: The GitHub token used to create an authenticated client | ||
default: ${{ github.token }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is automatic, but also necessary since it can't be extracted below.
a66856f
to
ecd18fe
Compare
ecd18fe
to
977411a
Compare
const token = core.getInput("github-token"); | ||
if (!token) { | ||
core.error( | ||
"GITHUB_TOKEN is not set. Please ensure the job has the necessary permissions. Attempting to run unauthenticated.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
grabbing the token is automatic (unless overriden). We can work without it, we just hit lower rate limits.
This allows us much more flexibility and ease of testing.